Skip to content

Conversation

characat0
Copy link
Contributor

@characat0 characat0 commented Sep 28, 2024

This PR aims to enable support for logging artifacts into remote servers.

Since mlflow v2, servers use proxy artifact stores by default. We can use the artifact_uri field of the mlflow run to identify server that support this feature and perform a PUT request to upload the artifacts.

@pebeto
Copy link
Member

pebeto commented Nov 2, 2024

@characat0 thanks for your contribution. I performed some research and it's interesting that the artifact API is hidden from users. I'm preparing a new major update to this package in this branch. If you are able, please adapt your changes to that specific branch.

Interesting info: https://github.com/mlflow/mlflow/blob/master/mlflow/server/handlers.py#L2168

@ablaom

@pebeto pebeto self-requested a review November 2, 2024 16:04
@pebeto pebeto added the enhancement New feature or request label Nov 2, 2024
@characat0
Copy link
Contributor Author

Given that the PR you mentioned is quite big, do you think it's better to wait until it's merged and then create another PR?
Or do you prefer to add them into the existing PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants